This PR resolves a couple of different issues in the repository, namely #66, #67, #61, #58, and #18. Each of these issues have a dedicated branch so you can see the changes originating from each issue independent of each other. After merging these, this PR also runs clang-format on the relevant source files and refactors around warnings a bit.

• HCal veto input collection is not configurable  #18 See https://github.com/LDMX-Software/Hcal/tree/18-hcal-veto-input-collection-is-not-configurable
  This just adds configurable collection names for the hcal veto processor
• Uninitialized entries in HcalVeto #58 See https://github.com/LDMX-Software/Hcal/tree/58-uninitialized-entries-in-hcalveto
  This adds a default value for the maxPE hit in the hcal veto collection with all values set to negative numbers. This isn't ideal of course, but signalling values is easier to spot than junk/uninitialized memory.
• Floating point values in integer conditions table  #66 See https://github.com/LDMX-Software/Hcal/tree/66-floating-point-values-in-integer-conditions-table
  This makes the rounding of the gain in the hcal trig primitives conditions explicit in the python configuration
• Hcal reconstruction performance significantly pessimized by TOA correction #67 See https://github.com/LDMX-Software/Hcal/tree/67-hcal-reconstruction-performance-significantly-pessimized-by-toa-correction
  This just marks the TGraph we use to do TOA correction as sorted so ROOT can use a binary search rather than linearly scanning through the entire graph

Finally¸this PR does the following refactorings

• Mark destructors for classes with virtual functions virtual
• Make trivial destructors and constructors defaulted
• Remove or mark unused variables as TODO/maybe_unused
• Patch a minor bug that clang's warnings caught

/home/einarelen/ldmx/ldmx-sw/Hcal/include/Hcal/HcalRawDecoder.h:347:23: warning: & has lower precedence than ==; == will be evaluated first [-Wparentheses]
            (w >> 15) & packing::utility::mask<1> == 1;
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/einarelen/ldmx/ldmx-sw/Hcal/include/Hcal/HcalRawDecoder.h:347:23: note: place parentheses around the '==' expression to silence this warning
            (w >> 15) & packing::utility::mask<1> == 1;
                      ^
                        (                             )

these were never used so the bug shouldn't have caused any problems

• Remove the #ifdef DEBUG parts from the raw decoder
• Add header files for all processors
  We had a bunch of processors that didn't have a corresponding header file. This made navigating the repo needlessly complicated and was hiding what processors we actually had in the repo. I'm more than happy to handle any merge conflicts that may arise from this refactoring :)
• clang-format